[Feature](show) add IP field display for show frontends/backends/broker commands and RestAPIs#60209
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-H: Total hot run time: 30842 ms |
TPC-DS: Total hot run time: 172048 ms |
ClickBench: Total hot run time: 27.03 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
cde916d to
3b5994a
Compare
|
run buildall |
TPC-H: Total hot run time: 31429 ms |
TPC-DS: Total hot run time: 173486 ms |
ClickBench: Total hot run time: 26.78 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
3b5994a to
c6fefed
Compare
|
run buildall |
c6fefed to
3ec00e7
Compare
TPC-H: Total hot run time: 30845 ms |
TPC-DS: Total hot run time: 172033 ms |
ClickBench: Total hot run time: 26.7 s |
2bd00f4 to
dc0d0ba
Compare
|
run buildall |
54e4dff to
c869fb3
Compare
|
run buildall |
TPC-H: Total hot run time: 30694 ms |
TPC-DS: Total hot run time: 172049 ms |
ClickBench: Total hot run time: 26.94 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
| public class BrokerMgr { | ||
| public static final ImmutableList<String> BROKER_PROC_NODE_TITLE_NAMES = new ImmutableList.Builder<String>() | ||
| .add("Name").add("Host").add("Port").add("Alive") | ||
| .add("Name").add("Host").add("IP").add("Port").add("Alive") |
There was a problem hiding this comment.
better append new column to the end for better compatibility.
it is the same for the other similar occurrences
There was a problem hiding this comment.
better append new column to the end for better compatibility. it is the same for the other similar occurrences
hello, I have already put this field to the end, and please review again~
c869fb3 to
8189a42
Compare
8189a42 to
aa3675c
Compare
aa3675c to
5c835d8
Compare
|
run buildall |
TPC-H: Total hot run time: 29916 ms |
TPC-DS: Total hot run time: 188265 ms |
ClickBench: Total hot run time: 28.74 s |
FE Regression Coverage ReportIncrement line coverage |
What problem does this PR solve?
Add IP field display for show frontends/backends/broker commands and RestAPIs
Check List (For Author)
Test
测试报告.docx
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)